Repository navigation
docs(rack): buy four-post support rails for the keyboard drawer and the Tiny tray - #1199
Merged
Merged
Conversation
…he Tiny tray Both trays were mounted on 2026-10-15 and wobble with all six ear screws tight: the keyboard drawer is a 10.8" slide with no rear support and the Tiny tray a 6.6" hinged shelf, and on a 29" four-post frame their rears rest on nothing. Two pairs of adjustable 1U four-post support rails join Everything still to buy, ten rows to eleven, with README's count following. hardware.md's two rows say why and footnote the parts; the move runbook lists the rails, fits them before each tray and fastens the tray's rear to them before anything is loaded above. Refs #919. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Approval recommended
The documentation changes are consistent across all five files, with no blocking issues identified.
0 open findings
What changed in this PR
Documents four-post support rails for the keyboard drawer and Tiny tray, aligning purchasing and installation guidance without changing the running lab.
Changes:
- Adds two pairs of support rails and updates the outstanding purchase count to eleven.
- Documents tray instability, part references, and installation steps.
- Records the decision in the changelog.
| File | Description |
|---|---|
| README.md | Updates the purchase count. |
| docs/runbooks/move-into-the-32u-rack.md | Adds rails to supplies and installation steps. |
| docs/roadmap.md | Adds the support-rail purchase. |
| docs/hardware.md | Explains rear support needs and links tray parts. |
| docs/changelog.md | Records the support-rail decision. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Chosen 2026-10-16 for a stated 10–31.5" mounting range, a 45 lb rating and a cabling crossbar the keyboard drawer's slider rears can tie to. The roadmap row names it the way the smaug shelf row names StarTech, the runbook's parts table and §6.1 say the same, and the changelog entry records the choice. Refs #919. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This was referenced Oct 16, 2026
Gerrrt
added a commit
that referenced
this pull request
Oct 16, 2026
#1199 landed on main as a squash while this branch carried its commits in original form. Both roadmap and runbook conflicts were the UPS rail row from #1201 against nothing on main's side, kept. README's purchase count was re-set to ten: git took main's eleven because this branch's ten matched the merge base, and the table has ten rows. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Docs only, no change to the running lab. Two pairs of adjustable 1U four-post support rails, spanning the frame's 29", join Everything still to buy for the keyboard drawer at U24 and
ifrit's Tiny tray at U18; ten rows become eleven and README's count follows.hardware.md's two rows say why (both are front-only mounts that wobble with all six ear screws tight) and gain footnotes naming the parts. The move runbook lists the rails in §0.1, fits them before each tray in §6, and §6.1 fastens each tray's rear to them before anything is loaded above. Changelog entry dated 2026-10-15.Why
Refs #919. Both trays were mounted on 2026-10-15 and move under hand pressure. The keyboard tray is a 10.8" slide with no rear support, which its reviewers also found; the Tiny tray is a 6.6" hinged shelf. On a 29" four-post frame their rears rest on nothing. The operator chose rails for both over replacing the Tiny tray with a fourth vented shelf.
Blast radius
None: documentation only.
secrets/*.sops.yamlVerification
python3 scripts/check_docs.py:docs OK — 158 Prometheus + 20 Loki rules, 8 dashboards, 156 panels, 12 assertions../scripts/lint.sh: every check PASS.make validate:all checks passed (5 skipped — NOT run, not verified). The five skips are the worktree ones: the offsite path refusal (CI covers it), unit syntax, installed timers, the sensitive stack's timers and the homelab-* units, none of which this checkout serves. Lint ran without--require-all; CI runs the hardened boots, the secret scan and the close-keyword check too.python3 scripts/check_close_keywords.py --texton this body: clean.make validatepassesDeployed to the lab and confirmed working (nothing to deploy)
Docs updated (
docs/, service README, ordocs/roadmap.md)🤖 Generated with Claude Code